Skip to content

fix(DBLink): leading/trailing whitespace when using icons (by mitosis update) #4310

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 16 commits into from
Jul 23, 2025

Conversation

nmerget
Copy link
Collaborator

@nmerget nmerget commented May 30, 2025

Proposed changes

closes #4072

Types of changes

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (fix on existing components or architectural decisions)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update (if none of the other choices apply)

Further comments

dependabot bot and others added 2 commits May 28, 2025 05:15
Bumps the builder-io group with 2 updates: [@builder.io/mitosis](https://github.com/BuilderIO/mitosis) and [@builder.io/mitosis-cli](https://github.com/BuilderIO/mitosis).


Updates `@builder.io/mitosis` from 0.9.2 to 0.11.0
- [Release notes](https://github.com/BuilderIO/mitosis/releases)
- [Commits](https://github.com/BuilderIO/mitosis/compare/@builder.io/[email protected][email protected]/[email protected])

Updates `@builder.io/mitosis-cli` from 0.9.2 to 0.11.0
- [Release notes](https://github.com/BuilderIO/mitosis/releases)
- [Commits](https://github.com/BuilderIO/mitosis/compare/@builder.io/[email protected][email protected]/[email protected])

---
updated-dependencies:
- dependency-name: "@builder.io/mitosis"
  dependency-version: 0.11.0
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: builder-io
- dependency-name: "@builder.io/mitosis-cli"
  dependency-version: 0.11.0
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: builder-io
...

Signed-off-by: dependabot[bot] <[email protected]>
@mfranzke mfranzke changed the title Fix mitosis update fix: mitosis update May 30, 2025
nmerget added 2 commits July 21, 2025 08:02
# Conflicts:
#	package-lock.json
#	package.json
#	packages/components/package.json
@nmerget nmerget marked this pull request as ready for review July 21, 2025 06:34
@nmerget nmerget requested a review from mfranzke as a code owner July 21, 2025 06:34
@nmerget nmerget moved this to 👀 In review in UX Engineering Team Backlog Jul 21, 2025
@mfranzke mfranzke self-assigned this Jul 21, 2025
mfranzke
mfranzke previously approved these changes Jul 21, 2025
@mfranzke mfranzke added the 🍄🆙improvement New feature or request label Jul 21, 2025
@nmerget nmerget enabled auto-merge (squash) July 21, 2025 06:45
@mfranzke mfranzke requested a review from Copilot July 21, 2025 16:27
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the Mitosis library from version 0.10.0 to 0.12.1 to fix related issues. The update enables Angular signals support and removes outdated workarounds that are no longer needed with the newer version.

Key changes include:

  • Updated Mitosis dependencies to version 0.12.1
  • Added Angular signals configuration with writeable properties for form components
  • Removed deprecated trackBy workarounds and lifecycle management code

Reviewed Changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated 9 comments.

Show a summary per file
File Description
package.json Updates Mitosis dependencies from 0.10.0 to 0.12.1
configs/angular/index.cjs Enables attribute passing in Angular configuration
Multiple component files Adds Angular signals metadata with writeable properties
scripts/post-build/angular.ts Removes deprecated workarounds and updates lifecycle handling
scripts/post-build/components.ts Removes trackBy function workarounds

@nmerget nmerget requested a review from mfranzke July 23, 2025 08:18
@nmerget nmerget merged commit 13cf340 into main Jul 23, 2025
70 checks passed
@nmerget nmerget deleted the fix-mitosis-update branch July 23, 2025 08:53
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in UX Engineering Team Backlog Jul 23, 2025
@mfranzke mfranzke changed the title fix: mitosis update fix: DBLink leading/trailing whitespace when using icons (by mitosis update) Jul 23, 2025
@mfranzke mfranzke changed the title fix: DBLink leading/trailing whitespace when using icons (by mitosis update) fix(DBLink): leading/trailing whitespace when using icons (by mitosis update) Jul 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍄🆙improvement New feature or request
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

DBLink leading/trailing whitespace when using icons
2 participants